-
Notifications
You must be signed in to change notification settings - Fork 81
Added GTT memory usage for GPU #556
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Hi, thanks a lot for the PR, I really appreciate it. :) I'm hesitant to merge this PR because of a couple of reasons, I'm afraid. First, I don't think GTT memory usage warrants such a prominent place in the UI. The GPU page is already the page with the most items, another huge graph item would be too much in my opinion. GTT memory usage is an AMD only feature (at least only AMD exposes this info for now), which means that for most users, this will show N/A anyway — another reason for it to be not as prominent. I think a better place would be somewhere incorporated in the Video Memory Usage string with no graph at all. Second, I'm sure most users don't know what "GTT" even means. Even I didn't know exactly what it meant before the issue was created. I'd like to keep most tech jargon out of Resources since I'd like Resources to be accessible for all users, whether they're into tech or not. Therefore, "GTT" would have to be replaced by a more understandable term. Third, the usage detail string would need to be fixed, relevant code starts here. GTT memory usage should probably not be displayed in the sidebar at all. If we could tackle these issues, we could introduce support for GTT usage after the release of Resources 1.9 (so we don't have to add any new translatable strings for now). :) |
Hi Nokyan, I agree that It maybe too much, I will make it a Video Memory Usage string, thank you very much for the feedback, I will get back to it when I find some free time! |
|
The "traditional" VRAM doesn't really matter for modern iGPUs anyway. It's only strictly required for some special buffers that need to be physically linear, e.g. for scanout. This is also why there isn't much point in increasing the VRAM allocation. Resources should simply display the sum of used memory in VRAM plus GTT. This is basically the only metric that actually has any value on iGPUs. |
This is very pragmatic to me. Also - it should be renamed from VRAM to "Graphics Memory". |


Added GTT memory usage for the GPU page that displays the graph of it with both used out of total GTT memory allocated per request of issue #523